Thanks Tim

I'll process it over the Christmas break.

Sorry, just been flat chat at work

Thanks again,
Vibul

On 12 December 2014 at 10:50, Tim Kellogg [email protected] wrote:

Fixes #102 #102

You can merge this Pull Request by running

git pull https://github.com/tkellogg/socko issue-102

Or view, comment on, or merge it at:

#105
Commit Summary

• Return 400 Bad Request when "Host" header is null

File Changes

• M
  socko-webserver/src/main/scala/org/mashupbots/socko/webserver/RequestHandler.scala
  https://github.com/mashupbots/socko/pull/105/files#diff-0 (8)

Patch Links:

• https://github.com/mashupbots/socko/pull/105.patch
• https://github.com/mashupbots/socko/pull/105.diff

—
Reply to this email directly or view it on GitHub
#105.

@veebs This looks good to me. We have been seeing failures when hit with HTTP/1.0 which does not require host header. This leads to connections not getting responses, timing out, and holding onto threads which eventually can kill servers hosting socko.